Skip to content

feat: Support for hiding implementation signature#286

Merged
pawamoy merged 3 commits into
mainfrom
bartosz/hide-implementation-signature
Jul 3, 2025
Merged

feat: Support for hiding implementation signature#286
pawamoy merged 3 commits into
mainfrom
bartosz/hide-implementation-signature

Conversation

@johnslavik

Copy link
Copy Markdown
Member

Closes #213.

Work is in progress as I'm investigating the test failures.

@johnslavik johnslavik changed the title Add support for hiding implementation signature feat: Add support for hiding implementation signature Jul 2, 2025
@johnslavik johnslavik changed the title feat: Add support for hiding implementation signature feat: Support for hiding implementation signature Jul 2, 2025
@pawamoy

pawamoy commented Jul 2, 2025

Copy link
Copy Markdown
Member

Tests seem to pass now 😄?

@johnslavik

Copy link
Copy Markdown
Member Author

Yeah, apparently. @pawamoy did you take a look at snapshots?

@pawamoy

pawamoy commented Jul 2, 2025

Copy link
Copy Markdown
Member

Not yet!

@johnslavik johnslavik marked this pull request as ready for review July 3, 2025 13:57
@johnslavik johnslavik requested a review from pawamoy July 3, 2025 13:57

@pawamoy pawamoy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK LGTM! Thanks ❤️

@pawamoy pawamoy merged commit d3b35e1 into main Jul 3, 2025
45 of 53 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feature: Option to hide implementation signature in case of overloads

2 participants